Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat(ui) System Stats, CPU RAM and GPU Usage #7005

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Feat(ui) System Stats, CPU RAM and GPU Usage #7005

wants to merge 10 commits into from

Conversation

mickr777
Copy link
Contributor

@mickr777 mickr777 commented Oct 2, 2024 •
edited
Loading

Summary

This PR introduces functionality for fetching and displaying system statistics, including CPU, RAM, and GPU usage. Currently tested on Windows with NVIDIA GPU

Animation

  • leaving as draft until devs can test/review it, also need ideas/suggestions/location (Current Location is for testing)
  • Added Placeholders for AMD and MPS Gpus
    (I have neither, so someone else will need to complete that part of the code)
  • universal cpu and ram usage should be easy with psutil
  • But universal gpu usage is much harder

Related Issues / Discussions

Dev Chat on discord
https://discord.com/channels/1020123559063990373/1049495067846524939/1289000704693899387

QA Instructions

For testing, you may need to run:
pip install psutil

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog_

@github-actions github-actions bot added api python PRs that change python files services PRs that change app services frontend PRs that change frontend files labels Oct 2, 2024
@mickr777
Copy link
Contributor Author

mickr777 commented Oct 4, 2024 •
edited
Loading

Animation
Maybe something like this?

@mickr777 mickr777 marked this pull request as ready for review October 4, 2024 04:34
@mickr777 mickr777 marked this pull request as draft October 4, 2024 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psychedelicious psychedelicious Awaiting requested review from psychedelicious psychedelicious is a code owner

@blessedcoolant blessedcoolant Awaiting requested review from blessedcoolant blessedcoolant is a code owner

@maryhipp maryhipp Awaiting requested review from maryhipp maryhipp is a code owner

@hipsterusername hipsterusername Awaiting requested review from hipsterusername hipsterusername is a code owner

@brandonrising brandonrising Awaiting requested review from brandonrising brandonrising is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
api frontend PRs that change frontend files python PRs that change python files services PRs that change app services
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mickr777